-
-
Notifications
You must be signed in to change notification settings - Fork 224
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
markedパッケージのバージョンを1.0.0から最新の4.0.10へ更新 #1380
Conversation
✔️ Deploy Preview for js-primer ready! 🔨 Explore the source changes: 56028a9 🔍 Inspect the deploy log: https://app.netlify.com/sites/js-primer/deploys/61e5504b519e350007072f05 😎 Browse the preview: https://deploy-preview-1380--js-primer.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ありがとうございます!
一つだけ 2つコメントしました。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
あと
markedパッケージをインポートした
marked
関数は、Markdown文字列を引数にとり、HTML文字列に変換して返します。
を
markedパッケージをインポートして取得した
marked.parse
関数は、Markdown文字列を引数にとり、HTML文字列に変換して返します。
とかに変更が必要ですかね。
markedパッケージをインポートした`marked`関数は、Markdown文字列を引数にとり、HTML文字列に変換して返します。 |
他には marked
を関数として言及しているところはなさそうですかね。
https://deploy-preview-1380--js-primer.netlify.app/use-case/nodecli/md-to-html/
同ページ内にある
の marked ですが、こちらも |
ていねいに次みたいな感じですかね。
marked自体はパッケージを示しているので、どちらでも大丈夫そうですね。
微妙な助詞が気になるので、こうした方が分かりやすいですかね。 |
こちらの文章良いですね! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
ありがとうございます!
📝 default exportは消えたけど、named exportである |
fix #1377
md-to-htmlとrefactor-and-unittestで利用しているmarkedのバージョンを最新の4系へ更新しました。
それに伴って関数の呼び出しが
marked(html)
->marked.parse(html)
へ変更されたので置換して、testが通ることを確認しました。